Skip to content

Conversation

@davidsbatista
Copy link
Contributor

@davidsbatista davidsbatista commented Dec 3, 2025

Related Issues

Proposed Changes:

  • added delete_by_filter() method to delete documents matching filter criteria
  • added update_by_filter() method to update fields of documents matching filter criteria
  • made _normalize_metadata_index_fields a static method (it doesn't need instance state)
  • removed erroneous @patch decorator from test_init in Azure AI Search tests that caused the test to use a MagicMock instead of a real instance

How did you test it?

  • new integration tests added for delete_by_filter and update_by_filter (5 new test cases)
  • CI tests

Checklist

@github-actions github-actions bot added integration:azure-ai-search type:documentation Improvements or additions to documentation labels Dec 3, 2025
@davidsbatista davidsbatista changed the title adding updated/delete by filter + tests feat: adding updated/delete by filter + tests to AzureusAISearch Document Store Dec 5, 2025
@davidsbatista davidsbatista removed the type:documentation Improvements or additions to documentation label Dec 5, 2025
@davidsbatista davidsbatista marked this pull request as ready for review December 5, 2025 17:12
@davidsbatista davidsbatista requested a review from a team as a code owner December 5, 2025 17:12
@davidsbatista davidsbatista requested review from Amnah199 and removed request for a team December 5, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add update_by_filter() and delete_by_filter() operations to AzureAISearchDocumentStore

2 participants